Skip to content

Avoid Unsloth import in Megatron service#686

Draft
Kovbo wants to merge 1 commit into
mainfrom
kovbo/megatron-service-avoid-unsloth-import
Draft

Avoid Unsloth import in Megatron service#686
Kovbo wants to merge 1 commit into
mainfrom
kovbo/megatron-service-avoid-unsloth-import

Conversation

@Kovbo
Copy link
Copy Markdown
Collaborator

@Kovbo Kovbo commented May 20, 2026

Summary

  • stop importing ART's Unsloth training stack from art.megatron.service at module import time
  • add a local GC/CUDA cleanup helper for MegatronService
  • keep create_identity_lora usable in Megatron-only environments without requiring TRL/vLLM side imports

Testing

  • uv run --with ruff ruff check src/art/megatron/service.py
  • PYTHONPATH=/tmp/art-serverless-fix/src uv run python - <<'PY' from art.megatron.service import create_identity_lora print(create_identity_lora.__name__) PY
  • created identity LoRA for Qwen/Qwen3-30B-A3B-Instruct-2507 with rank 1

@Kovbo Kovbo requested a review from bradhilton May 20, 2026 21:37
@Kovbo Kovbo marked this pull request as draft May 20, 2026 21:37
@Kovbo Kovbo requested review from FurtherAI and removed request for FurtherAI and bradhilton May 20, 2026 21:37
@Kovbo Kovbo force-pushed the kovbo/megatron-service-avoid-unsloth-import branch from 26edd52 to a2019a6 Compare May 21, 2026 00:10
@Kovbo Kovbo changed the title Avoid Unsloth import in Megatron service Fix Megatron service imports and Qwen3 identity LoRA export May 21, 2026
@Kovbo Kovbo force-pushed the kovbo/megatron-service-avoid-unsloth-import branch from a2019a6 to 8014898 Compare May 21, 2026 18:47
@Kovbo Kovbo changed the title Fix Megatron service imports and Qwen3 identity LoRA export Avoid Unsloth import in Megatron service May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant